Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

1346 symbolic link between our projects and the library 1 #1008

Merged

Conversation

alecarn
Copy link
Collaborator

@alecarn alecarn commented Sep 11, 2023

Rearchitect the app structure to allow a symlink between our project and our packages library igo2-lib.
See this PR for more information

@alecarn alecarn requested a review from pelord September 11, 2023 18:06
"input": "./node_modules/@igo2/core/locale/",
"output": "./locale/libs_locale",
"ignore": ["**/*.core.json"]
},
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Replace the scripts locale-copy-from-igo2-core

"sourceMap": true
"sourceMap": true,
"paths": {
"@igo2/*": ["node_modules/@igo2/*", "../../packages/*/src/public_api"]
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Workaround, when we refer to the symbolic link, we need to provide the source code for Typescript module resolution. We fallback on the last url if it exists.

"start-pwa": "ng build --configuration pwa --output-path ./dist/pwa && npm run serve.pwa",
"build": "ng build",
"build.prod": "ng build --configuration production",
"link.start": "ng serve --configuration developpement-link",
"link.build": "ng build --configuration production-link",
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

link.start vs start.link?

@alecarn alecarn merged commit c494d56 into next Oct 31, 2023
@alecarn alecarn deleted the 1346-symbolic-link-between-our-projects-and-the-library-1 branch October 31, 2023 18:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants